Skip to content

refactor: add yaml process system top level#1367

Merged
frodehk merged 6 commits intomainfrom
refactor/yaml-process-system-top-level
Feb 4, 2026
Merged

refactor: add yaml process system top level#1367
frodehk merged 6 commits intomainfrom
refactor/yaml-process-system-top-level

Conversation

@frodehk
Copy link
Copy Markdown
Contributor

@frodehk frodehk commented Feb 3, 2026

Refactors the YAML structure by introducing a top-level for PROCESS_SYSTEMS and adding PROCESS_SIMULATIONS under installation. Both optional.

Type of Work

  • Patch: X.Y.Z+1. NEGLIGIBLE visible changes, does not change input or output - OR changes behaviour. Use chore:, refactor: etc
  • Minor: X.Y+1.Z. Minor changes, might ADD new input (YAML), or other backwards-compatible changes. Use feat:, fix:
  • Major: X+1.Y.Z. Major and most likely BREAKING changes, wo. backwards compatibility, or removing temporary backwards compatibility functionality. Use ! or BREAKING:.

See here (internal): https://github.com/equinor/ecalc-internal/discussions/1044

Have you remembered and considered?

  • IF FEAT: I have remembered to update documentation
  • IF FIX OR FEAT: I have remembered to update manual changelog (docs/drafts/next.draft.md)
  • IF BREAKING: I have remembered to update migration guide (docs/docs/migration_guides/)
  • IF BREAKING: I have committed with BREAKING: in footer or ! in header
  • I have added tests (if not, comment why)
  • I have used conventional commits syntax (if you squash, make sure that conventional commit is used)
  • I have included the Github issue nr in the footer!

What is this PR all about?

What else did you consider?

Between the lines?

@frodehk frodehk self-assigned this Feb 3, 2026
@frodehk frodehk requested a review from a team as a code owner February 3, 2026 12:58
@frodehk frodehk marked this pull request as draft February 3, 2026 13:54
@frodehk frodehk marked this pull request as ready for review February 4, 2026 09:39
Comment thread src/libecalc/presentation/yaml/yaml_keywords.py Outdated
@frodehk frodehk force-pushed the refactor/yaml-process-system-top-level branch from 03d9ad0 to f691dd9 Compare February 4, 2026 11:55
@frodehk frodehk merged commit 18f5388 into main Feb 4, 2026
22 checks passed
@frodehk frodehk deleted the refactor/yaml-process-system-top-level branch February 4, 2026 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants